Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: upgraded versions.tf to include minor bumps from tpg v5 #118

Merged
merged 7 commits into from
Jan 8, 2024

Conversation

g-awmalik
Copy link
Contributor

No description provided.

@g-awmalik g-awmalik requested a review from a team as a code owner October 6, 2023 23:30
@apeabody apeabody changed the title chore: upgraded versions.tf to include minor bumps from tpg v5 fix: upgraded versions.tf to include minor bumps from tpg v5 Oct 16, 2023
@@ -20,11 +20,11 @@ terraform {

google = {
source = "hashicorp/google"
version = ">= 3.53, < 5.0"
version = ">= 3.53, < 6"
Copy link

@smaftoul smaftoul Oct 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would we block to version 6 ?
We don't know what's included in version 6 yet, and the module may still work.
Currently, it's "< 5.0" , and it's blocking terraform setups to update to provider v5, it's the only terraform google module that I use that does this.

Copy link

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label Dec 26, 2023
@github-actions github-actions bot closed this Jan 3, 2024
@apeabody apeabody reopened this Jan 3, 2024
@apeabody apeabody removed the Stale label Jan 3, 2024
@bharathkkb
Copy link
Member

@g-awmalik are the test failures related or can we merge this?

@apeabody
Copy link
Contributor

apeabody commented Jan 8, 2024

@g-awmalik are the test failures related or can we merge this?

I just merged #123 cleanly, let's see if we now get a green run. If not, likely an incompatibility with TPG 5 which needs to be investigated.

@apeabody
Copy link
Contributor

apeabody commented Jan 8, 2024

@bharathkkb @g-awmalik all green now :)

@apeabody apeabody merged commit 14fcdf3 into master Jan 8, 2024
6 checks passed
@apeabody apeabody deleted the chore/tpg-bump branch January 8, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants